feat: support new field server_certificate_rotation_mode for SQL instances #16072
Conversation
modular-magician
added
the
awaiting-approval
|
Hello! I am a robot. Tests will require approval from a repository maintainer to run. Googlers: For automatic test runs see go/terraform-auto-test-runs. @zli82016, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
modular-magician
added
service/sqladmin-cp
and removed
awaiting-approval
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_sql_database_instance" "primary" {
settings {
ip_configuration {
enable_private_path_for_google_cloud_services = # value needed
}
}
}
Missing doc report (experimental)The following resources have fields missing in documents.
|
Tests analyticsTotal tests: 137 Click here to see the affected service packages
Action takenFound 65 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
mmv1/third_party/terraform/services/sql/resource_sql_database_instance.go.tmpl
Outdated
Show resolved
Hide resolved
mmv1/third_party/terraform/services/sql/resource_sql_database_instance_test.go.tmpl
Outdated
Show resolved
Hide resolved
modular-magician
added
the
awaiting-approval
modular-magician
removed
the
awaiting-approval
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_sql_database_instance" "primary" {
settings {
ip_configuration {
enable_private_path_for_google_cloud_services = # value needed
}
}
}
Missing doc report (experimental)The following resources have fields missing in documents.
|
Tests analyticsTotal tests: 137 Click here to see the affected service packages
Action takenFound 31 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
mmv1/third_party/terraform/services/sql/resource_sql_database_instance.go.tmpl
Outdated
Show resolved
Hide resolved
wuchris-g
force-pushed
the
auto-rotation
branch
from
8e60c5b to
14dfbf2
Compare
modular-magician
added
the
awaiting-approval
modular-magician
removed
the
awaiting-approval
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_sql_database_instance" "primary" {
settings {
ip_configuration {
enable_private_path_for_google_cloud_services = # value needed
}
}
}
Missing doc report (experimental)The following resources have fields missing in documents.
|
Tests analyticsTotal tests: 137 Click here to see the affected service packages
Action takenFound 34 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
zli82016
left a comment
zli82016
left a comment
There was a problem hiding this comment.
🔴 Tests failed during RECORDING mode:
TestAccSqlDatabaseInstance_useCasBasedServerCa [Error message] [Debug log]
TestAccSqlDatabaseInstance_useInternalCaByDefault [Error message] [Debug log]
| Check: resource.ComposeTestCheckFunc( | ||
| resource.TestCheckResourceAttr(resourceName, "settings.0.ip_configuration.0.server_ca_mode", "GOOGLE_MANAGED_CAS_CA"), | ||
| resource.TestCheckResourceAttr(resourceName, "settings.0.ip_configuration.0.server_ca_pool", ""), | ||
| resource.TestCheckResourceAttr(resourceName, "settings.0.ip_configuration.0.server_certificate_rotation_mode", "AUTOMATIC_ROTATION_DURING_MAINTENANCE"), |
There was a problem hiding this comment.
Is this check in the wrong place?
The test TestAccSqlDatabaseInstance_useCasBasedServerCa failed
Error recording tests:
error running go: exit status 1
stdout:
=== RUN TestAccSqlDatabaseInstance_useCasBasedServerCa
=== PAUSE TestAccSqlDatabaseInstance_useCasBasedServerCa
=== CONT TestAccSqlDatabaseInstance_useCasBasedServerCa
resource_sql_database_instance_test.go:3817: Step 1/2 error: Check failed: Check 1/2 error: Check 3/6 error: google_sql_database_instance.instance: Attribute 'settings.0.ip_configuration.0.server_certificate_rotation_mode' expected "AUTOMATIC_ROTATION_DURING_MAINTENANCE", got ""
3 participants
Uh oh!
There was an error while loading. Please reload this page.